Skip to content

refactor(github-actions): Update issue labeler to apply milestones#3600

Merged
josephperrott merged 1 commit intoangular:mainfrom
thePunderWoman:issue-milestone
Apr 10, 2026
Merged

refactor(github-actions): Update issue labeler to apply milestones#3600
josephperrott merged 1 commit intoangular:mainfrom
thePunderWoman:issue-milestone

Conversation

@thePunderWoman
Copy link
Copy Markdown
Contributor

This updates the issue labeler to also run and apply the needsTriage and Backlog milestones when appropriate. This should eliminate the need for the ngbot entirely.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces automatic milestone assignment to the issue labeling action. It updates the logic to handle both 'opened' and 'labeled' events, assigning the 'needsTriage' milestone for area labels and the 'Backlog' milestone for priority labels (P0-P5). A review comment points out an improvement opportunity regarding milestone persistence: the current implementation unconditionally overwrites existing milestones, which could accidentally remove manually assigned release milestones. It is suggested to only apply the milestone if the field is empty or if transitioning specifically from 'needsTriage' to 'Backlog'.

Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This updates the issue labeler to also run and apply the needsTriage and Backlog milestones when appropriate. This should eliminate the need for the ngbot entirely.
@thePunderWoman thePunderWoman added the action: merge The PR is ready for merge by the caretaker label Apr 10, 2026
@josephperrott josephperrott merged commit 3a72c83 into angular:main Apr 10, 2026
13 checks passed
@josephperrott
Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants